Skip to content

Conversation

cdce8p
Copy link
Contributor

@cdce8p cdce8p commented Jun 10, 2025

Proposed change

typing.ByteString and collections.abc.ByteString will be removed in Python 3.14.
AFAICT the actual data type is bytes anyway, so just use that instead.
The alternative would be the full type bytes | bytearray | memoryview.

Originally added in #3644 by @abmantis.

https://docs.python.org/3.13/library/typing.html#typing.ByteString

Additional information

Checklist

  • The changes are tested and work correctly
  • pre-commit checks pass / the code has been formatted using Black
  • Tests have been added to verify that the new code works

Copy link

codecov bot commented Jun 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.24%. Comparing base (5f1f14b) to head (aac44fd).
Report is 1 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #4119      +/-   ##
==========================================
- Coverage   91.24%   91.24%   -0.01%     
==========================================
  Files         338      338              
  Lines       10919    10918       -1     
==========================================
- Hits         9963     9962       -1     
  Misses        956      956              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@TheJulianJES TheJulianJES added the code quality Improvement to code quality label Jun 16, 2025
Copy link
Collaborator

@TheJulianJES TheJulianJES left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@TheJulianJES TheJulianJES merged commit 07e835b into zigpy:dev Jun 16, 2025
9 checks passed
@cdce8p cdce8p deleted the replace-bytesting branch June 16, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code quality Improvement to code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants